Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release PR for 2.1.0 #222

Merged
merged 48 commits into from
Apr 15, 2024
Merged

Release PR for 2.1.0 #222

merged 48 commits into from
Apr 15, 2024

Conversation

nerdstrike
Copy link
Collaborator

No description provided.

mgcam and others added 30 commits February 23, 2024 11:36
All PacBio mlwh records are inspected, the ID is generated
from scratch and compared to the one stored in the database.
Added a scripts to validate ID generator
Update PacBioProductMetrics definition to include new per-product metrics
The descripton of the 'get_qc_states_by_id_product_list' method
in npg_langqc.db.helper.qc says that when sequencing qc states
are requested "it is guaranteed that the list of QcState objects
has only one member". However, the implementation did not warrant
this. It was possible to get an empty list of qc states.
Replaced multiple calls to retrieve qc states for
a list of products.

To simplify the logic, made this retrieval unconditional.
Whether the wells are likely to have any associated
qc state or not, an attempt to get qc states is made.
Where sequencing qc state for a single product is needed,
it is more efficient to use `get_qc_state_for_product`
method since it executes one SQL query, no additional
filtering is involved.
... for some models in order to simplify instantiation
of the objects.
Incorrect column name in mlwh
Refined the semantics of the 'study_names' field.
The function does a search on a table row, does not
take any other arguments. The table class of this row
is a natural home for this type of function.
The whole db is dropped and recreated at the point of
creating a session factory.
nerdstrike and others added 18 commits March 22, 2024 10:43
PacBioWellSummary as a response model for well summaries
mouseOver event for the well button displays a tooltip
with names of studies.
Study-aware well button colour scheme.
This state will be assigned to wells when the
outcome of the QC assessment depends on liasing
with a third party. The existing 'On hold' state
will be used for flagging internal investigations.

As fas as QC statuses and workflow is concerned,
the new state is handled in the same way as the
existing 'On hold' state.
Display study names and highlight BIOSCAN stady
Introduced 'On hold external' QC state.
Switched to light theme for the well button
Fixed an error in the db migration script
Updated the Copyright year.
Added release instructions to README.
Updated the CHANGELOG.
@nerdstrike nerdstrike merged commit e6e9dff into master Apr 15, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants